Skip to content

Conversation

@tomhatzer
Copy link
Contributor

Implements #675 and looks currently like this:

Articles:
image

Threads:
image

Replies:
image

@driesvints
Copy link
Member

@tomhatzer oh man, I think this escalated into much more than we intended 😅

What we basically wanted was to add a new nullable updated_by column that links to a user ID. Then if the updated_at is older than created_at, show "updated at X" and if there's a user ID, swow "updated at X by Y". No need for a new model etc 😅

UI wise it all looks good 👍 I also like your wording more than the above.

@tomhatzer
Copy link
Contributor Author

@driesvints hahahaha 😁 I didn't know what was expected so I did what felt okay for me. If it's too much, closing the PR won't cause any harm.

@driesvints
Copy link
Member

Yeah think we're just going to keep it simpler. Thanks though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants